Skip to content

Fix iOS UI test case crash#768

Merged
Kyle-Ye merged 2 commits intomainfrom
k-branch-2
Feb 8, 2026
Merged

Fix iOS UI test case crash#768
Kyle-Ye merged 2 commits intomainfrom
k-branch-2

Conversation

@Kyle-Ye
Copy link
Collaborator

@Kyle-Ye Kyle-Ye commented Feb 7, 2026

No description provided.

@github-actions github-actions bot added bug Something isn't working crash platform: iOS test Test related issue P1 Medium priority labels Feb 7, 2026
@augmentcode
Copy link

augmentcode bot commented Feb 7, 2026

🤖 Augment PR Summary

Summary: Adds a UIKit shim safe-wrapper for the private UIWindow._windowHostingScene API and switches UIHostingViewBase to use it, preventing iOS UI test crashes caused by unsafe perform/Unmanaged handling.

Notes: The wrapper returns nil when the selector isn’t available, keeping sceneActivationState guarded without runtime crashes.

🤖 Was this summary useful? React with 👍 or 👎

augmentcode[bot]
augmentcode bot reviewed Feb 7, 2026
View reviewed changes
Copy link

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.

@codecov
Copy link

codecov bot commented Feb 7, 2026 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 13.14%. Comparing base (b034c16) to head (69a82eb).
⚠️ Report is 1 commits behind head on main.

❗ There is a different number of reports uploaded between BASE (b034c16) and HEAD (69a82eb). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (b034c16) HEAD (69a82eb)
2 1
Additional details and impacted files 
@@             Coverage Diff             @@
##             main     #768       +/-   ##
===========================================
- Coverage   26.21%   13.14%   -13.08%     
===========================================
  Files         649      619       -30     
  Lines       40199    36740     -3459     
===========================================
- Hits        10540     4830     -5710     
- Misses      29659    31910     +2251

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Kyle-Ye Kyle-Ye merged commit 317a16d into main Feb 8, 2026
5 of 9 checks passed
@Kyle-Ye Kyle-Ye deleted the k-branch-2 branch February 8, 2026 04:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@augmentcode augmentcode[bot] augmentcode[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

bug Something isn't working crash P1 Medium priority platform: iOS test Test related issue

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Kyle-Ye